Skip to content

Conversation

@EinarSnorrason
Copy link
Contributor

Currently the action-helper file swallows all error messages, which makes it very difficult for users to debug issues that can occur in those functions. This PR adds error logging so errors from this module are displayed in the github action logs, making debugging easier.

@EinarSnorrason EinarSnorrason changed the title Updated action-helper.ts to log error messages fix: Updated action-helper.ts to log error messages Jul 5, 2024
@thc202 thc202 closed this Sep 26, 2025
@thc202 thc202 reopened this Sep 26, 2025
@psiinon
Copy link
Member

psiinon commented Sep 26, 2025

Logo
Checkmarx One – Scan Summary & Details2c7f2921-e89d-4792-b41e-72c6c2ebac0d

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

@thc202 thc202 changed the title fix: Updated action-helper.ts to log error messages chore: Inlcude error messages when logging Sep 26, 2025
@thc202 thc202 changed the title chore: Inlcude error messages when logging chore: Include error messages when logging Sep 26, 2025
Also use error instead of log.

Signed-off-by: EinarSnorrason <[email protected]>
@thc202
Copy link
Member

thc202 commented Sep 26, 2025

Thank you!

@psiinon psiinon merged commit 640dbd7 into zaproxy:master Sep 26, 2025
7 checks passed
@zapbot
Copy link

zapbot commented Sep 29, 2025

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants